Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Project Management Automation: Derive due date from previous milestone #20851

Merged
merged 2 commits into from
May 1, 2020

Conversation

aduth
Copy link
Member

@aduth aduth commented Mar 12, 2020

Blocked by (merges to): #20850
Supersedes #20189

This pull request seeks to update the existing addMilestone task to avoid using constants for reference versions upon which the next milestone due date is calculated. As seen in #20189, these constants are fragile due to exceptions in release cycle. The changes here propose to use the previous milestone due date as the reference date. To support this, included are additional improvements to ensure that the milestone results from the GitHub REST API can be correctly paginated.

This currently merges to the branch of #20850; the intention being to use the types-checking as a developer experience improvement for working with the Octokit REST API library, since the addition of types enables auto-completion in supported IDEs.

Testing Instructions:

Existing tests have been updated:

npm run test-unit packages/project-management-automation

@aduth aduth added [Type] Enhancement A suggestion for improvement. [Package] Project management automation /packages/project-management-automation labels Mar 12, 2020
@aduth aduth requested a review from noisysocks March 12, 2020 21:37
@github-actions
Copy link

github-actions bot commented Mar 12, 2020

Size Change: +8.87 kB (1%)

Total Size: 825 kB

Filename Size Change
build/annotations/index.js 3.62 kB -1 B
build/api-fetch/index.js 4.08 kB +1 B
build/autop/index.js 2.82 kB +2 B (0%)
build/block-directory/index.js 6.6 kB +374 B (5%) 🔍
build/block-editor/index.js 107 kB +770 B (0%)
build/block-library/editor-rtl.css 7.08 kB +52 B (0%)
build/block-library/editor.css 7.08 kB +54 B (0%)
build/block-library/index.js 115 kB +1.18 kB (1%)
build/block-library/style-rtl.css 7.22 kB +77 B (1%)
build/block-library/style.css 7.23 kB +81 B (1%)
build/block-serialization-spec-parser/index.js 3.1 kB +1 B
build/blocks/index.js 48.1 kB -3 B (0%)
build/components/index.js 179 kB +68 B (0%)
build/compose/index.js 6.66 kB -1 B
build/data/index.js 8.44 kB +3 B (0%)
build/edit-navigation/index.js 4.05 kB +512 B (12%) ⚠️
build/edit-post/index.js 28.1 kB +539 B (1%)
build/edit-post/style-rtl.css 12.2 kB +26 B (0%)
build/edit-post/style.css 12.2 kB +27 B (0%)
build/edit-site/index.js 11.4 kB +497 B (4%)
build/edit-site/style-rtl.css 5.18 kB +74 B (1%)
build/edit-site/style.css 5.18 kB +75 B (1%)
build/edit-widgets/index.js 7.77 kB +276 B (3%)
build/editor/index.js 44.3 kB +683 B (1%)
build/editor/style-rtl.css 5.07 kB +1.8 kB (35%) 🚨
build/editor/style.css 5.08 kB +1.81 kB (35%) 🚨
build/format-library/index.js 7.63 kB +4 B (0%)
build/list-reusable-blocks/index.js 3.13 kB +1 B
build/notices/index.js 1.79 kB -1 B
build/nux/index.js 3.4 kB +1 B
build/plugins/index.js 2.56 kB -114 B (4%)
build/primitives/index.js 1.5 kB +1 B
build/redux-routine/index.js 2.85 kB -2 B (0%)
build/rich-text/index.js 14.8 kB +3 B (0%)
build/server-side-render/index.js 2.67 kB -6 B (0%)
build/shortcode/index.js 1.69 kB -1 B
build/token-list/index.js 1.28 kB -1 B
build/url/index.js 4.02 kB -1 B
build/warning/index.js 1.14 kB +1 B
ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.02 kB 0 B
build/blob/index.js 620 B 0 B
build/block-directory/style-rtl.css 760 B 0 B
build/block-directory/style.css 761 B 0 B
build/block-editor/style-rtl.css 10.2 kB 0 B
build/block-editor/style.css 10.2 kB 0 B
build/block-library/theme-rtl.css 683 B 0 B
build/block-library/theme.css 685 B 0 B
build/block-serialization-default-parser/index.js 1.88 kB 0 B
build/components/style-rtl.css 16.9 kB 0 B
build/components/style.css 16.9 kB 0 B
build/core-data/index.js 11.4 kB 0 B
build/data-controls/index.js 1.29 kB 0 B
build/date/index.js 5.47 kB 0 B
build/deprecated/index.js 772 B 0 B
build/dom-ready/index.js 568 B 0 B
build/dom/index.js 3.1 kB 0 B
build/edit-navigation/style-rtl.css 485 B 0 B
build/edit-navigation/style.css 485 B 0 B
build/edit-widgets/style-rtl.css 4.67 kB 0 B
build/edit-widgets/style.css 4.66 kB 0 B
build/editor/editor-styles-rtl.css 428 B 0 B
build/editor/editor-styles.css 431 B 0 B
build/element/index.js 4.65 kB 0 B
build/escape-html/index.js 733 B 0 B
build/format-library/style-rtl.css 502 B 0 B
build/format-library/style.css 502 B 0 B
build/hooks/index.js 2.13 kB 0 B
build/html-entities/index.js 622 B 0 B
build/i18n/index.js 3.56 kB 0 B
build/is-shallow-equal/index.js 710 B 0 B
build/keyboard-shortcuts/index.js 2.51 kB 0 B
build/keycodes/index.js 1.94 kB 0 B
build/list-reusable-blocks/style-rtl.css 226 B 0 B
build/list-reusable-blocks/style.css 226 B 0 B
build/media-utils/index.js 5.29 kB 0 B
build/nux/style-rtl.css 616 B 0 B
build/nux/style.css 613 B 0 B
build/priority-queue/index.js 789 B 0 B
build/viewport/index.js 1.84 kB 0 B
build/wordcount/index.js 1.18 kB 0 B

compressed-size-action

@aduth aduth changed the base branch from add/project-management-automation-types to master April 3, 2020 16:06
@aduth aduth force-pushed the update/project-management-derive-due-date branch from 17bee20 to 3003d67 Compare April 3, 2020 16:07
Copy link
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It makes sense to approach it this way 👍

@aduth aduth force-pushed the update/project-management-derive-due-date branch from 0677682 to d58b673 Compare April 23, 2020 20:07
@aduth aduth force-pushed the update/project-management-derive-due-date branch from d58b673 to af7f5ef Compare April 29, 2020 21:01
@aduth aduth merged commit a9dd93e into master May 1, 2020
@aduth aduth deleted the update/project-management-derive-due-date branch May 1, 2020 19:15
@github-actions github-actions bot added this to the Gutenberg 8.1 milestone May 1, 2020
@aduth
Copy link
Member Author

aduth commented May 1, 2020

Logging looks good at the merge automation:

https://github.com/WordPress/gutenberg/runs/637302498

add-milestone: Fetching current milestone
add-milestone: Fetching `package.json` contents
add-milestone: Current plugin version is 8.0
add-milestone: Creating 'Gutenberg 8.1' milestone, due on 2020-05-11T07:00:00.000Z
add-milestone: Milestone already exists, proceeding with assignment
add-milestone: Fetching all milestones
add-milestone: Adding issue #20851 to milestone #120

Notably, regarding #20851 (comment), I marked the Gutenberg 8.0 milestone as closed immediately prior to merging this pull request, and it was still able to discover it and compute the correct next due date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Project management automation /packages/project-management-automation [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants